Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add deposit and delete operation to GVG in signer and fix a few issue #1258

Merged
merged 4 commits into from
Dec 5, 2023

Conversation

alexgao001
Copy link
Contributor

@alexgao001 alexgao001 commented Nov 30, 2023

Description

This Pr mainly adds the deposit function and a few fixes.

  1. The manager keep track of GVG usage and make a deposit if available store size insufficient. Or delete it if it has been empty for over 1 day.
  2. Sort the all SP by id when retrieved from Chain API, so that in a new GVG's creation, secondary SP in ascending order.
  3. Querying GVG via meta API in the virtual group manager loop, and extend to every 5 second.
  4. Add retry to report task after uploading is done incase task is lost, and missing log for failure.

Rationale

tell us why we need these changes...

Example

add an example CLI or API response...

Changes

Notable changes:

  • add each change in a bullet point here
  • ...

Potential Impacts

  • add potential impacts for other components here
  • ...

@alexgao001 alexgao001 changed the title feat: add deposit to signer and fix a few issue feat: add deposit and delete operation to GVG in signer and fix a few issue Dec 4, 2023
@alexgao001 alexgao001 merged commit 4142018 into bnb-chain:develop Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants